Adwaita: properly fix places sidebar

Forgot to treat treeview background... Some code refactoring in
the process.

diff --git a/gtk/resources/theme/Adwaita/_colors.scss b/gtk/resources/theme/Adwaita/_colors.scss
index a612caedb5b365a131351c455ce5f6dd51dd9bce..200969a115174bc65ee11b49af0fe53be8bfe08b 100644 (file)
--- a/gtk/resources/theme/Adwaita/_colors.scss
+++ b/gtk/resources/theme/Adwaita/_colors.scss
@@ -26,6 +26,8 @@ $osd_bg_color: #2e3436;
 $osd_borders_color: transparentize(black, 0.3);
 $osd_outer_borders_color: transparentize(white, 0.9);
 
+$sidebar_bg_color: lighten($bg_color,5%);
+
 $tooltip_borders_color: $osd_outer_borders_color;
 
 //insensitive state derived colors
@@ -41,3 +43,4 @@ $backdrop_fg_color: mix($fg_color, $backdrop_bg_color, 50%);
 $backdrop_insensitive_color: if($variant == 'light', darken($backdrop_bg_color, 15%), lighten($backdrop_bg_color, 15%));
 $backdrop_borders_color: mix($borders_color, $bg_color, 90%);
 $backdrop_dark_fill: mix($backdrop_borders_color, $backdrop_bg_color, 35%);
+$backdrop_sidebar_bg_color: lighten($backdrop_bg_color,5%);

diff --git a/gtk/resources/theme/Adwaita/_common.scss b/gtk/resources/theme/Adwaita/_common.scss
index f387f917b68e81f780478af11f9d876cfd09da95..e7905b5d21cf2f4546a7be5a878d742734eb4e26 100644 (file)
--- a/gtk/resources/theme/Adwaita/_common.scss
+++ b/gtk/resources/theme/Adwaita/_common.scss
@@ -2702,8 +2702,8 @@ GtkFileChooserDialog {
 
 .sidebar {
   border: none;
-  $_sidebar_bg: lighten($bg_color,5%);
-  background-color: $_sidebar_bg;
+  background-color: $sidebar_bg_color;
+
   .scrollbar {
     &.trough {
       background-color: darken($bg_color,10%);
@@ -2711,14 +2711,17 @@ GtkFileChooserDialog {
       border-width: 2px;
       border-color: transparent;
     }
+
     &.slider {
-      background-color: $_sidebar_bg;
+      background-color: $sidebar_bg_color;
       &:hover { background-color: lighten($bg_color, 10%); }
     }
   }
+
   &:backdrop {
-    background-color: lighten($backdrop_bg_color,5%);
+    background-color: $backdrop_sidebar_bg_color;
   }
+
   &:selected {
     background-color: $selected_bg_color;
   }
@@ -2726,16 +2729,24 @@ GtkFileChooserDialog {
 
 // Places sidebar is a special case, since the view here have to look like chrome not content, so we override text color
 GtkPlacesSidebar.sidebar .view {
+
   color: $fg_color;
+  background-color: transparent;
+
   &:backdrop { color: $backdrop_fg_color; }
+
   &:selected {
     color: $selected_fg_color;
+    background-color: $selected_bg_color;
+
     &:backdrop { color: if($variant == 'light', $backdrop_base_color, $selected_bg_color); }
   }
+
   .separator,
   .separator:backdrop { @extend .separator; }
 }
 
+
 .sidebar-item {
   padding: 10px 4px;
   > .label {
@@ -2748,6 +2759,11 @@ GtkPlacesSidebar.sidebar .view {
   }
 }
 
+
+/*********
+ * Paned *
+ *********/
+
 GtkPaned { // this is for the standard paned separator
 
   -GtkPaned-handle-size: 1; // sets separator width

diff --git a/gtk/resources/theme/Adwaita/gtk-contained-dark.css b/gtk/resources/theme/Adwaita/gtk-contained-dark.css
index 191cb65cb75468ef729f7bbde85c56ab5806da27..47dd882849e61d04577e5de0c0f6cf34bd22de5b 100644 (file)
--- a/gtk/resources/theme/Adwaita/gtk-contained-dark.css
+++ b/gtk/resources/theme/Adwaita/gtk-contained-dark.css
@@ -4052,11 +4052,13 @@ GtkFileChooserDialog .dialog-action-box {
     background-color: #215d9c; }
 
 GtkPlacesSidebar.sidebar .view {
-  color: #eeeeec; }
+  color: #eeeeec;
+  background-color: transparent; }
   GtkPlacesSidebar.sidebar .view:backdrop {
     color: #939695; }
   GtkPlacesSidebar.sidebar .view:selected {
-    color: #ffffff; }
+    color: #ffffff;
+    background-color: #215d9c; }
     GtkPlacesSidebar.sidebar .view:selected:backdrop {
       color: #215d9c; }
 
@@ -4068,6 +4070,9 @@ GtkPlacesSidebar.sidebar .view {
   .sidebar-item.needs-attention > .label {
     background-size: 6px 6px, 0 0; }
 
+/*********
+ * Paned *
+ *********/
 GtkPaned {
   -GtkPaned-handle-size: 1;
   margin: 0 8px 8px 0; }

diff --git a/gtk/resources/theme/Adwaita/gtk-contained.css b/gtk/resources/theme/Adwaita/gtk-contained.css
index fa6e203fa36b69206871296c1d264d448052030c..0d4c33e9cea45457bcc6a707ff49a9e8dbfe9c3a 100644 (file)
--- a/gtk/resources/theme/Adwaita/gtk-contained.css
+++ b/gtk/resources/theme/Adwaita/gtk-contained.css
@@ -4213,11 +4213,13 @@ GtkFileChooserDialog .dialog-action-box {
     background-color: #4a90d9; }
 
 GtkPlacesSidebar.sidebar .view {
-  color: #2e3436; }
+  color: #2e3436;
+  background-color: transparent; }
   GtkPlacesSidebar.sidebar .view:backdrop {
     color: #8d9091; }
   GtkPlacesSidebar.sidebar .view:selected {
-    color: #ffffff; }
+    color: #ffffff;
+    background-color: #4a90d9; }
     GtkPlacesSidebar.sidebar .view:selected:backdrop {
       color: white; }
 
@@ -4229,6 +4231,9 @@ GtkPlacesSidebar.sidebar .view {
   .sidebar-item.needs-attention > .label {
     background-size: 6px 6px, 0 0; }
 
+/*********
+ * Paned *
+ *********/
 GtkPaned {
   -GtkPaned-handle-size: 1;
   margin: 0 8px 8px 0; }